Skip to content

Misc SSD1327 and SSD1363 improvements #94316

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

VynDragon
Copy link
Contributor

greyscale -> grayscale
+const
ssd1363 lut -> ssd1327 style lut

LUT table to rom instead of ram

Signed-off-by: Camille BAUD <[email protected]>
josuah
josuah previously approved these changes Aug 11, 2025
Copy link
Contributor

@josuah josuah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code quality goes up again for SSDxxx drivers!

british english -> american english

Signed-off-by: Camille BAUD <[email protected]>
Model on ssd1327

Signed-off-by: Camille BAUD <[email protected]>
josuah
josuah previously approved these changes Aug 11, 2025
JarmouniA
JarmouniA previously approved these changes Aug 11, 2025
Copy link
Contributor

@danieldegrasse danieldegrasse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No problem with changing grey-> gray, but do we have any documentation on the preferred spelling? Both are correct.

If we are changing property names, we should add this to the migration guide.

@VynDragon
Copy link
Contributor Author

No problem with changing grey-> gray, but do we have any documentation on the preferred spelling? Both are correct.

If we are changing property names, we should add this to the migration guide.

Yes #87921 (comment)

Comment on lines +93 to +97
grayscale-enhancement:
type: boolean
description: Enable greyscale enhancement, undocumented.
description: Enable grayscale enhancement, undocumented.

greyscale-table:
grayscale-table:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change needs to be mentioned in migration guide as its requiring downstream users to make changes to their code

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@VynDragon VynDragon dismissed stale reviews from JarmouniA and josuah via febd94a August 13, 2025 10:31
notice change of property name

Signed-off-by: Camille BAUD <[email protected]>
@zephyrbot zephyrbot added the Release Notes To be mentioned in the release notes label Aug 13, 2025
@zephyrbot zephyrbot requested review from cfriedt and jhedberg August 13, 2025 10:32
Copy link

Please retry analysis of this Pull-Request directly on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Display Release Notes To be mentioned in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants